自动化代码审核 — PR #520

由 Agent 自动审核，最终合并决定由维护者把关。

[HIGH] _name_match 在 listener 真正运行前就被 drop — NameOwnerChanged 永远收不到

位置：openless-all/app/src-tauri/src/linux_fcitx.rs，start_dictation_signal_listener 内新增的 NameOwnerChanged 注册块。

let _name_match = match conn.add_match(fcitx_rule, move |...| { ... }) {
    Ok(m) => m,
    ...
};
// 紧接着进入 retry 循环和 `loop {}` 事件循环

dbus crate 的 add_match 返回一个 MsgMatch guard — guard drop 时 match 即被注销。_name_match 是局部变量，作用域在外层 block 结束时就被析构，而真正处理事件的 loop {} 还在后面。结果：fcitx5 重启后自动重新同步快捷键这条核心能力实际上不会工作，尽管源码看上去逻辑齐全。

修复：把 _name_match 的绑定挪到紧邻 loop {} 之前的位置，或者改用一个生命周期能贯穿整个事件循环的容器（例如 Vec<MsgMatch> 放在 thread-local 或外层作用域）。

[HIGH] 仓库里提交了一个空的 libopenless.so 占位文件

位置：openless-all/app/src-tauri/linux-fcitx5-plugin/libopenless.so（zero-byte，blob SHA e69de29b）

CI 在打包前会用真实编译产物覆盖这个路径，所以 release 流程是好的。但任何开发者在本地 clone 后跑 Linux build，如果没执行完整的 CI 插件构建步骤，ensure_plugin_installed 就会把这个空 .so 拷到 ~/.local/lib/fcitx5/，fcitx5 加载时静默失败，没有可用的错误信息。

建议二选一：

1. 从源码树里删掉这个文件，仅依赖 CI 的 cp 步骤；或
2. 改成 .gitkeep 哨兵 + 启动时检查文件大小，发现是空文件直接报错而不是拷过去。

[MEDIUM] emit_capsule Linux 分支：两次 spawn 的 retry thread 与 LAST_AUX 之间存在窄竞态

位置：coordinator.rs emit_capsule 内的 Linux 块。

was_none 触发时无条件 spawn 一个 300ms 后重试的线程。如果状态在 300ms 内 Recording → Idle → Recording，两个 retry 线程都会看到相同的 LAST_AUX 文本，都通过 current.as_deref() != Some(&text) 守卫，结果是 两次 DBus 调用。建议引入 AtomicBool cancel 标志，或者用单个 debounce task 替代每次都 spawn。

[MEDIUM] setAuxDown 中的 savedIc_ 存在窄析构窗口

位置：openless-all/scripts/linux-fcitx5-plugin/openless.cpp，setAuxDown / clearAuxDown。

注释说"事件处理线程和 DBus 处理线程都是 fcitx5 主事件循环，无竞态"——对从按键事件触发的路径是对的。但现在 setAuxDown 也会由 Rust 侧 std::thread::spawn 出来的线程经 DBus 触发，进入 fcitx5 event loop 后，理论上仍可能在 if (!ic) null-check 与 ic->inputPanel() 解引用之间，被同一 event loop 迭代中 dispatch 的 InputContextDestroyed 事件清空 savedIc_。flushUI() 能排空已有事件但挡不住其后到达的 destroy 事件。建议把 IC 访问 + panel 更新包进 instance_->callLater 或 foreachFocused 闭包里。

[MEDIUM] bailian.rs 去重：sentence_id == 0 的最终句会被静默丢弃

位置：openless-all/app/src-tauri/src/asr/bailian.rs 第 +69 行。

if is_sentence_final && sentence_id > 0 {
    st.final_segments.insert(sentence_id, trimmed.to_string());
}

sentence_id 通过 unwrap_or(0) 兜底。如果 API 偶尔返回 end_time > 0 但缺 sentence_id 的最终句，旧代码会 push 到 Vec，新代码会丢。至少应在 is_sentence_final && sentence_id == 0 时打一条 warn 日志。

[LOW] retry 循环阻塞了 listener 线程，听 NameOwnerChanged 与 30s 同步串行了

10 × 3s 同步等待结束之前 NameOwnerChanged 是不在跑的（即便修了上面的 drop bug 也存在 3s 窗口）。可接受，但和 HIGH#1 联动，建议先注册 match 再开始 retry。

[LOW] release-tauri.yml 旧注释

PR 已更新文案，无需处理。

[LOW] coordinator.rs 内联 use std::sync::Mutex;

#[cfg(target_os = "linux")] 块内的局部 use，与文件顶部 import 风格不一致，可选清理。

结论：WARNING — 2 个 HIGH 都直接影响 PR 自身宣称的功能（fcitx5 重启后自动同步 + AppImage 自动装插件），建议合并前修复。MEDIUM/LOW 不阻塞但值得跟进。

🤖 Generated by code-reviewer agent.

自动化复审 — PR #520（commits 5208499 + de5c55b 之后）

Follow-up review after author addressed my earlier review.

✅ HIGH #1 已修复

52084999 中 _name_match 的绑定已挪到外层作用域，生命周期覆盖了完整的 loop { conn.process(...) }。callback 内部增加了手工守卫 if name != "org.fcitx.Fcitx5" { return true; }，替代了 dbus crate 不支持的 arg0 过滤。逻辑正确。

附加改进：thread::sleep 从 callback 内挪到 spawn 出来的线程里，顺便修了 reconnect 时事件循环被阻塞的潜在问题，赞。

❌ HIGH #2 仍未修复

openless-all/app/src-tauri/linux-fcitx5-plugin/libopenless.so 在当前 PR branch 中仍是 0 字节（wc -c 返回 0）。

ensure_plugin_installed 的注释更新为"启动时总会覆盖"，但仓库里 bundle 进 AppImage 的 .so 本身还是空的。任何 Linux AppImage 构建仍会把零字节 .so 拷到 ~/.local/lib/fcitx5/，fcitx5 加载时静默失败。

建议二选一：

1. 把这个文件从仓库里删掉，完全依赖 CI 在打包前编译并 cp 真实产物；或
2. 启动时检查 .so 文件大小，0 字节直接报错而不是拷贝，避免误以为安装成功。

✅ MEDIUM #2 已修复

de5c55b 给 clear_aux_down 加了对称的状态守卫，在 spawn 出来的线程内重新检查 LAST_AUX.is_none()，与 set_aux_down 的守卫对齐。能正确防止陈旧的 clear 越过新的 set。

结论：HIGH #2 仍阻塞合并 — Linux AppImage 那条主线还是会出问题。

🤖 Generated by code-reviewer agent.